License is not being detected

Describe the bug
Running scancode for the following github package (https://github.com/stleary/JSON-java/tree/20230227), it doesn't detect the license inside the pom.xml file (https://github.com/stleary/JSON-java/blob/20230227/pom.xml)
System configuration

Which version of ScanCode.io are you running? 32.2.1
Are you running the app using Docker? yes

Expected behavior
Expect to detect the Public Domain license.

@cgi-ricardo thanks for the report, this is a bug indeed.
Note that we are able to detect the license correctly in the context of a package, where we extract license statements in the specific context of the package manifest and then scan that statement for licenses.

But we also scan the package manfiest files (here the pom.xml) with a license scanner without using the context, as a whole file, and there we are getting false positives:
    "license_clues": [
      {
        "license_expression": "cc0-1.0",
        "license_expression_spdx": "CC0-1.0",
        "from_file": "JSON-java-20230227/pom.xml",
        "start_line": 36,
        "end_line": 42,
        "matcher": "3-seq",
        "score": 59.09,
        "matched_length": 13,
        "match_coverage": 59.09,
        "rule_relevance": 100,
        "rule_identifier": "cc0-1.0_200.RULE",
        "rule_url": "https://github.com/nexB/scancode-toolkit/tree/develop/src/licensedcode/data/rules/cc0-1.0_200.RULE",
        "matched_text": "    <licenses>\n        <license>\n            <name>Public Domain</name>\n            <url>https://github.com/stleary/JSON-java/blob/master/LICENSE</url>\n            <distribution>repo</distribution>\n        </license>\n    </licenses>",
        "matched_text_diagnostics": "licenses>\n        <license>\n            <name>Public Domain</name>\n            <url>[https]://[github].[com]/[stleary]/[JSON]-[java]/[blob]/[master]/[LICENSE]</url>\n            <distribution>repo</distribution>\n        </license>\n    </licenses>"
      }
    ],

Note that we are also detecting that there is something wrong with the match and hence considering this as a clue instead of a detection, and so not reporting this in the resource license_expression too.
But we can do better:

Add specific rules to detect this as a public-domain license
Do more to combine file and package license detections to remove ambiguity in these cases.

This detection issue is actually present in scancode-toolkit, so moving it there. Also attaching the scancode-toolkit scan results.
JSON-java.json


Add benchmarks

What was wrong?
We currently do not have benchmarks, which means we don't really know how well the event bus scales under load.
We'd like to get various metrics for different scenarios e.g.

pushing n number of events to m processes
same as above but with different size of payload
...

How was it fixed?
Adds some basic benchmarks. Still incomplete and not not polished.
Cute Animal Picture

@pipermerriam This can be given a full review now. The code isn't free of flaws (e.g. the benchmark not shutting down cleanly is really annoying) but I consider it good enough as a first shot.
I'd also recommend to take a look at the actual output. Does it make sense to you, are there other metrics you'd like to see? What is your feeling about scalability based on these numbers? I guess it would also be extremely interesting to create another benchmark that uses the Python proxy primitives to put these numbers better into perspective (e.g can we expect perf gains or losses by migrating to Lahja?)
My feeling based on these numbers is that it scales well enough under moderate load. E.g if the overall traffic that we'll have on the EventBus is less than 100 events per second (can be simulated with --throttle 0.01) than I think we'll not have a problem.
Here we are pushing 100 events per second, each event with 1 MB payload into 5 processes. The average time spent from sending to receiving is about 0.007 seconds which strikes me as fast enough.
I kinda want to believe that we would not need more than 100 events per second load on the EventBus in Trinity :)
$ python scripts/perf_benchmark.py --num-processes 5 --num-events 1000 --throttle 0.01 --payload-bytes 1000000
                                                                    +++Globals+++
|Consumer processes |   Total time   | Total aggegated time  | Propagated events  |Received events | Propagated EPS |  Received EPS  |
|         5         |    10.73539    |       40.49281        |        1000        |      5000      |     93.150     |    465.749     |
                                                                +++Process Details+++
|      Process      |Processed Events|   First sent   | Last received  |    Fastest     |    Slowest     |      AVG       | Total duration | Total aggregated time |
|    consumer_2     |      1000      |  11:52:09.792  |  11:52:20.435  |    0.00383     |    0.02183     |    0.00807     |    10.64307    |        8.06907        |
|    consumer_4     |      1000      |  11:52:09.792  |  11:52:20.435  |    0.00360     |    0.02562     |    0.00881     |    10.64265    |        8.80900        |
|    consumer_0     |      1000      |  11:52:09.792  |  11:52:20.443  |    0.00269     |    0.01716     |    0.00751     |    10.65036    |        7.51061        |
|    consumer_1     |      1000      |  11:52:09.792  |  11:52:20.438  |    0.00323     |    0.01605     |    0.00770     |    10.64594    |        7.69999        |
|    consumer_3     |      1000      |  11:52:09.792  |  11:52:20.441  |    0.00366     |    0.02842     |    0.00840     |    10.64931    |        8.40413        |

Raw time metrics are useful but inverting them to something like events-per-second is a metric that humans can often interpret more easily.

EPS = Events per second. I just used that to make the columns more narrow.


pixelated fallback font

http://maps.stamen.com/terrain/beta/#7/32.301/21.083

http://maps.stamen.com/terrain/beta/#12/32.4639/130.1870

I don't know if there's much we can do with the second one near Hondo, but I wonder why the first one near Benghazi is happening?
Do we need a few more fallback fonts before we end up at that really pixelated one (which I think is unifont)?

Similarly pixelated in Chinese:
http://maps.stamen.com/terrain/beta/#14/39.9869/116.3405
Must be a way to do better...? :)

I don't know if there's much we can do with the second one near Hondo

Both the labels near Hondo are similarly pixelated.
PT Sans has a very limited range of characters. You could try adding DejaVu to the list to improve the selection. If you did that and the Noto CJK fonts you'd get most of east Asia.
If you want more comprehensive font lists, I suggest looking at OpenStreetMap Carto and the past discussion, since we've tended to be the cutting edge for an open map that renders labels around the entire world.


illegal memory access

Hello,
Thank you very much for roviding the C3D version of pytorch. I am trying to extract features for another dataset, which contains around 1748 frames in one video. While doing so I am facing an issue as following:-
RuntimeError: CUDA error: an illegal memory access was encountered
My input size is after passing through the function get_sport_clip() is torch.Size([1, 3, 1748, 112, 112])
I was wondering if you could kindly assist me in solving this issue.
Thank you very much once again.

Hi and thanks for interest.
You can't do that.
That's because this architecture is tailored to take as input 16-frames clips.
Theoretically (i.e. with infinite gpu memory) you could compute features for such input shapes up to pool5. After that, the reshape at line 59 of C3D_model.py would result in a different dimension w.r.t. the subsequent linear layer, causing an error.
A common practice in those cases is to compute descriptors for 16-frames clips. In your case that would result in 1748-16 descriptors. But it depends on your application.
Hope this helps,
D

Thank you for your feedback.
Just to summarize, what you are trying to say is that the model is trained to input 16 sequential frames and computes its corresponding features for these 16 frames. This would be computed for
total frames / 16 times?
Hope this is correct?
Thank you

Just to summarize, what you are trying to say is that the model is trained to input 16 sequential frames and computes its corresponding features for these 16 frames.

Precisely.

This would be computed for total frames / 16 times?

It depends on what you feed to the model. If you build 16 frames clip striding one frame at a time (x_0 to x_15, x_1 to x_16 and so on) it would result in total_frames - 16 descriptors. But again, it depends on you and what you want to do with such descriptors.
D

Cool...Thank you very much!


How to visualize the test results of shape2motion dataset?

Hi, according to the tutorial, I have realized the visualization of test results of the real word dataset, which depends on the input of depth images and RGB images. However, how to visualize the test results of the shape2motion dataset? Thank you very much for your help!

Hi, visualizing test results would be even easier. You just need to create a datamodule from the config file by running datamodule = hydra.utils.instantiate(config.datamodule), and get dataloader by dl = datamodule.test_dataloader(). Then you can roll out the data and pass them into the network as shown in the demo, and visualize the results using the same code.

Thank you very much for your help. You are so excellent! But can you show me what you modified? I don't know where to modify the code.

Hi, you can use this code to instantiate the datamodule with config:
datamodule = hydra.utils.instantiate(config.datamodule)
datamodule.setup()
dl = datamodule.test_dataloader()
it = iter(dl)

And this code to rollout the data:
sample = next(it)
for k, v in sample.items():
    if isinstance(v, torch.Tensor):
        sample[k] = v.to(device)

Hi, you can use this code to instantiate the datamodule with config:
datamodule = hydra.utils.instantiate(config.datamodule)
datamodule.setup()
dl = datamodule.test_dataloader()
it = iter(dl)

And this code to rollout the data:
sample = next(it)
for k, v in sample.items():
    if isinstance(v, torch.Tensor):
        sample[k] = v.to(device)

Thank you very much. With your help, I have completed this task!


Enable Security and Compliance tasks in our Release pipeline

Enables a series of tasks run against our release pipeline that validate the security and compliance status of our code in an automated fashion. These checks include:

Component Governance - (we had this one, it was moved to here) - Inventories open-source components used in our build
PREfast - C/C++ static analysis for common code errors and exploits
Policheck - Searches source code, comments, and text for words that could be sensitive legally, culturally, or geopolitically
Credscan - Looks for credentials left behind in the code/documents and build output files
BinSkim - Searches for common vulnerabilities in binaries
CheckCFlags - Validates that compile/link flags match the policies recommended by Windows engineering for inclusion into the OS product image
CFGCheck/XFGCheck - Validates that the CFG and/or XFG settings were enabled at compile and link time to guard against control flow attacks.

We're also required to run the SBOM one, but that was done in a separate PR and we're still pending the detectors being updated.
References

#11948 - Move from CFG to XFG once XFG task folks get back to me on it
#11949 - Enable bug filing for SecComp tasks
#11950 - Bulk process bugs filed by SecComp tasks
#11947 - Validate SBOM when checkers come online

Checklist

[x] - Fixes #10735
[x] - I work here
[x] - If it fits, it sits.

From the meeting, @DHowett wants me to pull the compliance one into a template of steps and then link it here.

/azp run


Add the functionality to change and reset a user's password

Hey, can i work on it?

Give me this issue, i can make it!

GFI @NicolasBiondini 👍

I just started and i'm triying to make it, but it's gonna be much simple if someone make a PR of the login functionality and the JWT.

I just started and i'm triying to make it, but it's gonna be much simple if someone make a PR of the login functionality and the JWT.

Yeah sure one PR is under review for the login functionality it will be merged soon.

Is this issue still open?

Is this issue still open?

Yeah sure are you up for it?

Yeah sure. How do you want to reset, via mail reset or just a classic reset interface?

Basically You need to create two apis one will be the protected api which will be used for changing the password of a user when he's logged in and for reset password api it will be the classic email reset api.

Ok, I'll do it!

Ok, I'll do it!

Go for it @DevParmar1


can't create service policy with @ identity name

this fails in zac and via cli
eugene@scruffy:~/work/scruffy-env$ ziti edge controller list identities
id: 1fc6b5aa-bc13-450b-beb5-b47d7068457c    name: hermes    type: Device    role attributes: {}
id: b9fba8ad-262c-4169-b4bb-658121bb7454    name: Default Admin    type: User    role attributes: {}
results: 1-2 of 2

eugene@scruffy:~/work/scruffy-env$ ziti edge controller create service-policy hermes-bind1 Bind -i @hermes -r '#all'
panic: error creating service-policies instance in Ziti Edge Controller at https://scruffy:1280. Status code: 400 Bad Request, Server returned: {"error":{"args":{"urlVars":{}},"cause":{"message":"no identities found with the given ids","field":"identityRoles","value":["hermes"]},"causeMessage":"the value '[hermes]' for 'identityRoles' is invalid: no identities found with the given ids","code":"INVALID_FIELD","message":"The field contains an invalid value","requestId":"b9624172-6e08-4438-a703-92bc63f02dcb"},"meta":{"apiEnrolmentVersion":"0.0.1","apiVersion":"0.0.1"}}

goroutine 1 [running]:
github.com/netfoundry/ziti-cmd/ziti/cmd/ziti/cmd/edge_controller.runCreateServicePolicy(0xc000297540, 0xc000167d60, 0xc000167d38)
        /home/eugene/work/github/ziti-cmd/ziti/cmd/ziti/cmd/edge_controller/create_service_policy.go:84 +0x606
github.com/netfoundry/ziti-cmd/ziti/cmd/ziti/cmd/edge_controller.newCreateServicePolicyCmd.func1(0xc00029e780, 0xc0002b1c20, 0x2, 0x6)
        /home/eugene/work/github/ziti-cmd/ziti/cmd/ziti/cmd/edge_controller/create_service_policy.go:54 +0x66
github.com/spf13/cobra.(*Command).execute(0xc00029e780, 0xc0002b1bc0, 0x6, 0x6, 0xc00029e780, 0xc0002b1bc0)
        /home/eugene/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:830 +0x29d
github.com/spf13/cobra.(*Command).ExecuteC(0xf8a0a0, 0x1006998, 0x0, 0x0)
        /home/eugene/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:914 +0x2fb
github.com/spf13/cobra.(*Command).Execute(...)
        /home/eugene/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:864
github.com/netfoundry/ziti-cmd/ziti/cmd/ziti/cmd.Execute()
        /home/eugene/work/github/ziti-cmd/ziti/cmd/ziti/cmd/cmd.go:77 +0x51
main.main()
        /home/eugene/work/github/ziti-cmd/ziti/cmd/ziti/main.go:24 +0x20

eugene@scruffy:~/work/scruffy-env$ ziti edge controller create service-policy hermes-bind1 Bind -i @1fc6b5aa-bc13-450b-beb5-b47d7068457c -r '#all'
0b31d769-8e3a-4e38-a368-e2e5474c21b3

eugene@scruffy:~/work/scruffy-env$

From the REST API perspective, this is working as intended. We had support for @name but it was agreed to only support @id and support for @name was removed. It can be added back in the clients (CLI/ZAC).


[TW#19013] aws_iot has a memory leak due to mutexes being created but never freed

Diagnosing a memory leak we're seeing in 2 of our apps with the heap tracing enabled. The app connects and disconnects from AWS periodically and we notice that we leak after each time it connects to AWS.
What I found was a very slow leak and xQueueGenericCreate() seems to be called each time.

11 allocations trace (300 entry buffer)

80 bytes (@ 0x3ffedfc8) allocated CPU 1 ccount 0x433e7330 caller 0x4008a342:0x4008a5a6:0x40123577:0x401225e8:0x40120d0c
0x4008a342: xQueueGenericCreate at /home/esp32/esp/esp-idf/components/freertos/./queue.c:2037
0x4008a5a6: xQueueCreateMutex at /home/esp32/esp/esp-idf/components/freertos/./queue.c:2037
0x40123577: aws_iot_thread_mutex_init at /home/esp32/esp/esp-idf/components/aws_iot/port/threads_freertos.c:37
0x401225e8: aws_iot_mqtt_init at /home/esp32/esp/esp-idf/components/aws_iot/aws-iot-device-sdk-embedded-C/src/aws_iot_mqtt_client.c:192
0x40120d0c: aws_iot_shadow_init at /home/esp32/esp/esp-idf/components/aws_iot/aws-iot-device-sdk-embedded-C/src/aws_iot_shadow.c:78

Ok, makes sense these mutexes are being created and dispose. So lets check if aws_iot_thread_mutex_destroy()  is called to free the mutex with a grep.

$ pwd
/home/esp32/esp/esp-idf/components/aws_iot/aws-iot-device-sdk-embedded-C/src
$ grep -r aws_iot_thread_mutex_ *
aws_iot_mqtt_client.c:          threadRc = aws_iot_thread_mutex_trylock(pMutex);
aws_iot_mqtt_client.c:          threadRc = aws_iot_thread_mutex_lock(pMutex);
aws_iot_mqtt_client.c:  return aws_iot_thread_mutex_unlock(pMutex);
aws_iot_mqtt_client.c:  rc = aws_iot_thread_mutex_init(&(pClient->clientData.state_change_mutex));
aws_iot_mqtt_client.c:  rc = aws_iot_thread_mutex_init(&(pClient->clientData.tls_read_mutex));
aws_iot_mqtt_client.c:  rc = aws_iot_thread_mutex_init(&(pClient->clientData.tls_write_mutex));
$ pwd
/home/esp32/esp/esp-idf/components/aws_iot
$ grep -r vSemaphoreDelete *
port/threads_freertos.c:    vSemaphoreDelete(pMutex->mutex);

So from my search, looks like the aws_iot port never frees the mutexes that it creates. This would explain the slow leak that we're seeing in our apps.
I did a test and moved aws_iot_shadow_init() so it is only init'd once at start up and this solved our memory leak issue. Unfortunately this produced a new bug but it is not ESP32 related (if you're curious, the bug is running out of client tokens due to this one line when calling aws_iot_shadow_register_delta(). The tokens are never zeroed until aws_iot_shadow_init() is called again).
I don't know what the solution is yet but I'm thinking of putting aws_iot_thread_mutex_destroy() into the disconnect logic and seeing if that fixes everything nicely (and properly).

@jaimeyu Thanks for reporting this issue. But it looks like something to do with AWS IoT SDK, probably you have already reported and acknowledged here, https://github.com/aws/aws-iot-device-sdk-embedded-C/issues/145. Whenever fix will be available, we will pull it in IDF. For now this issue can be closed here.

Thanks for looking into it. I agree, when I opened the bug, I was basing it out of my system path but I realized afterwards the esp IDF was using a sub git project.


typeahead still leaving spaces instead of deleting

Went back to delete the double a (yes, the MBP keyboard has driven me crazy thanks for asking), got a space.

/duplicate https://github.com/microsoft/vscode/issues/109704


CyclicPartitioner, RandomPartitioner, and KeyedPartitioner do not work properly.

Partitioners do not receive list of partitions parameter and as the result:
CyclicPartitioner always returns 0
KeyedPartitioner and RandomPartitioner fail

call client.refreshMetadata([topic1, topic2], callback) before sending.


Add shared inputs/outputs json for Pig Latin exercise

This is a proposed fix for exercism/todo#125.
The tests all take an input string and compare the output to an expected string, so the test cases all have the same structure.
Some languages provide a named test for each case, and some group them together into parameterized tests. I've included descriptions at both levels so that the generators can use one or both types of description.
What do you think?

Thank you, @suzannehamilton, this looks great.

Three minutes to merge. Amazing, thanks. :grin:

Haha, I was right in the middle of processing my GitHub notifications :) Just good timing is all!


How to add permissions to ranks?

Hi,
Im new to using better ban and I was wondering how to I add permissions to ranks?
Example: betterban.command.ban
I don't know what the command would be and would appreciate if u put up a guide for that as well.

BetterBan has nothing to do with your ranking system.

its not about my ranking system its about setting betterban permissions.
Lets say I want to give my admins permissions to use betterban. I need to go into Admin rank config and add the betterban permission but I don't know what the command would be. As of right only OP can use betterban.

The following Commands are having the default PocketMine-MP permissions:

Ban
IP-Ban
Pardon (Unban)
Pardon-IP (Unban-IP)

yes, but how do I assign them to certain ranks?
Example: Admin rank, permissions:

pocketmine.command.ban
pocketmine.comman.list
I wanna know what the command(example.command) would be in the config?

You can't edit the permissions if you mean that.

doesn't matter I figured it out

and I did it


Qasm exporter creates invalid Qasm for subcircuits with classical registers

Environment

Qiskit Terra version: 0.19.1
Python version: 3.8
Operating system: Ubuntu 18.04.6 LTS

What is happening?
Converting a circuit with a subcircuit to qasm fails, when this subcircuit contains also classical registers.
How can we reproduce the issue?
Run this:
from qiskit import QuantumCircuit, ClassicalRegister, QuantumRegister
qr = QuantumRegister(2, name='qr')
cr = ClassicalRegister(2, name='cr')
qc = QuantumCircuit(qr, cr, name='qc')
subcircuit = QuantumCircuit(qr, cr, name='subcircuit')
subcircuit.x(qr[0])
qc.append(subcircuit, qargs=qr, cargs=cr)
qc.measure(qr, cr)
qc.draw(fold=-1)

Output:
      ┌─────────────┐┌─┐
qr_0: ┤0            ├┤M├───
      │             │└╥┘┌─┐
qr_1: ┤1            ├─╫─┤M├
      │  subcircuit │ ║ └╥┘
cr_0: ╡0            ╞═╩══╬═
      │             │    ║
cr_1: ╡1            ╞════╩═
      └─────────────┘

Convert to qasm:
qc.qasm(formatted=True)

Output:
OPENQASM 2.0;
include "qelib1.inc";
gate subcircuit q0,q1 { x q0; }
qreg qr[2];
creg cr[2];
subcircuit qr[0],qr[1],cr[0],cr[1];
measure qr[0] -> cr[0];
measure qr[1] -> cr[1];

Now, if we want to reuse this qasm and load it:
qc = QuantumCircuit.from_qasm_str(qc.qasm())

We get the following error:
qiskit/circuit/quantumcircuit.py in from_qasm_str(qasm_str)
   2362         """
   2363         qasm = Qasm(data=qasm_str)
-> 2364         return _circuit_from_qasm(qasm)
   2365
   2366     @property

qiskit/circuit/quantumcircuit.py in _circuit_from_qasm(qasm)
   4695     from qiskit.converters import dag_to_circuit
   4696
-> 4697     ast = qasm.parse()
   4698     dag = ast_to_dag(ast)
   4699     return dag_to_circuit(dag)

qiskit/qasm/qasm.py in parse(self)
     51         with QasmParser(self._filename) as qasm_p:
     52             qasm_p.parse_debug(False)
---> 53             return qasm_p.parse(self._data)

qiskit/qasm/qasmparser.py in parse(self, data)
   1138     def parse(self, data):
   1139         """Parse some data."""
-> 1140         self.parser.parse(data, lexer=self.lexer, debug=self.parse_deb)
   1141         if self.qasm is None:
   1142             raise QasmError("Uncaught exception in parser; " + "see previous messages for details.")

ply/yacc.py in parse(self, input, lexer, debug, tracking, tokenfunc)
    331             return self.parseopt(input, lexer, debug, tracking, tokenfunc)
    332         else:
--> 333             return self.parseopt_notrack(input, lexer, debug, tracking, tokenfunc)
    334
    335

ply/yacc.py in parseopt_notrack(self, input, lexer, debug, tracking, tokenfunc)
   1118                             del symstack[-plen:]
   1119                             self.state = state
-> 1120                             p.callable(pslice)
   1121                             del statestack[-plen:]
   1122                             symstack.append(sym)

qiskit/qasm/qasmparser.py in p_unitary_op_2(self, program)
    704         """
    705         program[0] = node.CustomUnitary([program[1], program[2]])
--> 706         self.verify_as_gate(program[1], program[2])
    707         self.verify_reg_list(program[2], "qreg")
    708         self.verify_distinct([program[2]])

qiskit/qasm/qasmparser.py in verify_as_gate(self, obj, bitlist, arglist)
    158
    159         if g_sym.n_bits() != bitlist.size():
--> 160             raise QasmError(
    161                 "Gate or opaque call to '" + obj.name + "' uses",
    162                 str(bitlist.size()),

QasmError: "Gate or opaque call to 'subcircuit' uses 4 qubits but is declared for 2 qubits line 6 file "

What should happen?
I would have expected the exported qasm to be a valid qasm (aka to be read again) and also give me back the original exported circuit if I decide to store my circuit on a file as qasm and pass it to someone else.
Any suggestions?
I would have expected the following QASM:
qasm_expected = """
OPENQASM 2.0;
include "qelib1.inc";
gate subcircuit q0,q1 { x q0; }
qreg q[2];
creg c[2];
creg meas[2];
subcircuit q[0],q[1];
barrier q[0],q[1];
measure q[0] -> meas[0];
measure q[1] -> meas[1];
"""
qc = QuantumCircuit.from_qasm_str(qasm_expected)
qc.draw(fold=-1)

        ┌─────────────┐ ░ ┌─┐
   q_0: ┤0            ├─░─┤M├───
        │  subcircuit │ ░ └╥┘┌─┐
   q_1: ┤1            ├─░──╫─┤M├
        └─────────────┘ ░  ║ └╥┘
   c: 2/═══════════════════╬══╬═
                           ║  ║
meas: 2/═══════════════════╩══╩═
                           0  1

But this is also inaccurate, since in the original subcircuit might as well use the classical registers to perform some measurement, whereas with this qasm representation it is not possible to express this.
Thus, I am not sure on how we should proceed and I very curious to listen to your feedback on this.
Thanks in advance

There is no possible valid OpenQASM 2 programme for the circuit you're describing; there's no subroutine-like construct that can take classical parameters.  The error message could be better, but there is no way to produce valid OpenQASM 2 in this situation, because the object has no way of being represented.  The bug here is that the QASM 2 exporter should have rejected the circuit out-of-hand.  Probably the fix involves modifying the exporter step that tries to find the definition of each object in a circuit so that it goes through the following steps:

if a Gate instance, proceed as it currently does
if Barrier, Measure or Reset, output the specialist QASM 2 statements
if not a Gate (i.e. an unknown Instruction), then fail with a message saying that QASM 2 cannot represent non-unitary operations (except for the built-in measure and reset).

I haven't looked at the QASM 2 exporter code for a while, but I think it tries to export all Instruction instances currently, whereas it should fail on anything other than a special case or a gate, due to limitations in the QASM 2 language.
OpenQASM 3 can represent such instructions, but at the moment Terra's QASM 3 exporter is quite limited in the subroutine blocks it can support, since that language spec is still evolving, and so is Terra's capability to represent dynamic circuits.

I attach this bug report here, since closely related (instead of opening a new one)
Environment

Qiskit Terra version: 0.43.1 meta package, terra 0.24.1
Python version: 3.10
Operating system: docker continuumio/miniconda3

What is happening?
Exporting a circuit with a sub-circuit via append() and that uses measure() leads to an invalid qasm file, generating error when imported again.
How can we reproduce the issue?
Run this python script:
from qiskit import QuantumCircuit
q = QuantumCircuit(2)
q.initialize('00')
print(q.qasm(filename="my.qasm"))
round_trip = QuantumCircuit.from_qasm_str(q.qasm())

Produces this output and error:
Converting to qasm and back for circuit circuit-114
     ┌───┐     ┌─┐   ┌──────────────┐
q_0: ┤ H ├──■──┤M├───┤0             ├
     └───┘┌─┴─┐└╥┘┌─┐│              │
q_1: ─────┤ X ├─╫─┤M├┤1             ├
          └───┘ ║ └╥┘│  circuit-115 │
c_0: ═══════════╩══╬═╡0             ╞
                   ║ │              │
c_1: ══════════════╩═╡1             ╞
                     └──────────────┘
OPENQASM 2.0;
include "qelib1.inc";
gate circuit_115 q0,q1 { h q0; cx q0,q1; measure q0; measure q1; }
qreg q[2];
creg c[2];
h q[0];
cx q[0],q[1];
measure q[0] -> c[0];
measure q[1] -> c[1];
circuit_115 q[0],q[1],c[0],c[1];

Error near line 3 Column 15
Traceback (most recent call last):
  File "myfile.py", line 16, in <module>
    QuantumCircuit().from_qasm_str(circuit2.qasm())
  File "...qiskit/circuit/quantumcircuit.py", line 2529, in from_qasm_str
    return _circuit_from_qasm(qasm)
  File "...qiskit/circuit/quantumcircuit.py", line 4964, in _circuit_from_qasm
    ast = qasm.parse()
  File "...qiskit/qasm/qasm.py", line 53, in parse
    return qasm_p.parse(self._data)
  File "...qiskit/qasm/qasmparser.py", line 1137, in parse
    self.parser.parse(data, lexer=self.lexer, debug=self.parse_deb)
  File "...ply/yacc.py", line 333, in parse
    return self.parseopt_notrack(input, lexer, debug, tracking, tokenfunc)
  File "...ply/yacc.py", line 1120, in parseopt_notrack
    p.callable(pslice)
  File "...qiskit/qasm/qasmparser.py", line 397, in p_id_e
    raise QasmError("Expected an ID, received '" + str(program[1].value) + "'")
qiskit.qasm.exceptions.QasmError: "Expected an ID, received 'measure'"

The qasm contains also a second error leading to an error, thus the fact that circuit_115 is defined with 2 qubits, but then it is used with 4 arguments (see: https://github.com/Qiskit/qiskit-terra/issues/7750#issue-1163308223)
What should happen?
I would expect a valid qasm since the circuit is theoretically representable in qasm.
Any suggestions?
What about expanding the definition and injecting the basic instructions directly in the main code, without a separate definition, thus avoiding the problem with the measure and number of arguments?
This would clearly lead to a bigger qasm file, but it would be valid.

Thanks for the bug report - the second one is a duplicate of #8048.  The first one was fixed already by #10438, so I'll close this issue now.


Caching strategy for legendastv provider

Removing caching when listing archives

when an episode or a movie is released, you'll have a growing list of archives for the next couple of days. If we cache this URL (carrega_legendas_busca_filme) we might wait for a long time to get a subtitle that was added after your initial search. This page doesn't show an episode list. It shows subtitles list and caching it for days 3 days (EPISODE_EXPIRATION_TIME) is not appropriate.

Changing release names cache to 15 min

It's a common practice that the uploaders re-upload the archivers adding more releases to it. And that happens in a high frequency for the first couple of days after the release. This cache should be invalidated when the archive changes; or its expiration time should be short.

Related PR: #612
I've created issues for each PR that I have submitted to keep track and a open discussion since they are blocking me to use legendastv provider on my daily tasks. I'm really looking forward for the 2.0 release with all the great features that were added. Really good work :+1:

@ratoaq2 if you want I can add you to the SickRage/subliminal repo so we can combine efforts. That way we can iron out the details and get the better changes into SickRage a bit sooner (and tested a bit more before they are merged upstream). And won't have dupe PR's either =P


Upgrade terraform-provider-okta to v4.4.3

This PR was generated via $ upgrade-provider pulumi/pulumi-okta.

Upgrading terraform-provider-okta from 4.4.2  to 4.4.3.
Fixes #402

/release patch


Duplicate Audit Log Events

In the latest app router release we are seeing audit log event conflicts.  Identical logs are being created within the same millisecond.  This is usually caused by async functions referencing the same resource in different code paths.

Although related to #3668 these audit log conflicts should be easier to resolve and are removed from the scope of the Database Performance issue.

The getFullTemplateByID was being called multiple times in a single render causing identical audit logs.

I'll be refactoring to use the client side Template Store as the source of truth as much as possible where the current function is called.


Quick Link view sample

Category

[X] New Sample

What's in the Pull Request?
Quick Link gallery view samples.

Thanks for the awesome and beautiful samples @watana2 😍😍✨
After checking the samples and README.md, I noticed one point that needs to be corrected and one point that I think could be improved. Could you please check the following points and make any necessary corrections? If you have any questions or concerns, feel free to leave a comment!
Correction Request
The sample screenshot in the README.md includes company logos. The use of company logos may involve copyright issues or require adherence to specific guidelines. Therefore, I would appreciate it if you could avoid using the logos and replace the screenshot if possible.
Suggestion for Improvement
When I clicked on the link, the image opened. When I think of a link, I have the image of a site opening instead of an image. So, as an example, how about creating a column called “URL column” and making the link destination the site set in that URL column?

*Correction Request: Updated.

Thanks for the update!

Additional columns: Initially, multiple columns were used, but they weren't found to be useful over time.

I understand! I also realized that the image path needs to be set in the URL of the hyperlink column, while the link destination should be set in the Display text. (I had mistakenly set the image path in both the URL and the Display text, so the image would open when I clicked the link.)
However, I believe this way of using hyperlink columns is not common. Typically, the URL of the hyperlink column is set to the link destination, and the text of that link is set in the Display text. This might cause confusion for those using this sample.
So, as a sample, could we change the current hyperlink column to use the ImageURL column (column storing the URL of the image, single line of text) and the URL column (column storing the URL of the link destination, single line of text) instead of the Icon column in the current hyperlink column? (If you are unable to take the time to make corrections, it is possible for me to make the correction.)

Also, this view formatting sample also contains a column formatting sample (title-format.json). Since this is not a view formatting sample, can it be removed or created as a new column formatting sample?
Sorry for the many requests...😣

Hi @watana2 ! Sorry for the delay.
I did the merge, please check it out!

https://github.com/pnp/List-Formatting/tree/master/view-samples/picture-quick-links
https://github.com/pnp/List-Formatting/tree/master/column-samples/hyperlink-display-url

Also, when merging, I made the following modifications.

There was a column formatting sample mixed in with the view formatting sample in the submission. Therefore, the column formatting sample was created separately.
I think that alternative text is originally intended to describe the content of an image for users who may not be able to view the image or use a screen reader. Therefore, I still felt uncomfortable with displaying the alternate text of the hyperlink column (Icon column) with the URL of the image as the link title. Therefore, as a sample, I divided the hyperlink column into two, one for the link and the other for the picture. (Sorry...)
Added a note to the README.
Other minor adjustments were made.

If you have any questions or concerns, please comment!
Once again, thank you for the beautiful and cool samples @watana2 😍😍😍✨

These samples will also be posted in the Sample Solution Gallery. Please also check it out.
⚠ It will take a little time for the sample to be reflected in the Sample Solution Gallery. (probably up to 1 day).

If you are interested, please register for the following program. When you register, you will get a digital badge!
M365 Platform & Power Platform Community Recognition Program
⚠Even if you register, it will not be distributed immediately. It will take some time before it is distributed.


Fix #53 by using a Reflect.construct based inheritance mechanism

i’m quite sure Reflect will decrease performance, i would check. this is not solution that we trying to get here and with with worse output - no reason to use. but need to check how bad is it?!
Sent from iPhone

On Jun 29, 2018, at 05:57, haveyaseen<EMAIL_ADDRESS>wrote:
You can view, comment on, or merge this pull request online at:
https://github.com/tommikaikkonen/redux-orm/pull/213
Commit Summary
add test reproducing #53
run lint after build in travis
fix #53 by using a Reflect.construct based inheritance mechanism
File Changes
M .travis.yml (3)
A jest.es5.config.js (9)
M package.json (1)
M src/Session.js (13)
A src/test/es5Test.js (24)
Patch Links:
https://github.com/tommikaikkonen/redux-orm/pull/213.patch
https://github.com/tommikaikkonen/redux-orm/pull/213.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, it does indeed seem to decrease performance :disappointed: Averages of 10 measurements:

Creating 10000 objects
Looking up 10000 objects
Adding 2500 relations
Performing 500 queries
Removing 1000 relations

extends
0.3319s
0.2015s
2.5097s
2.0617s
2.6538s

Reflect.construct
0.4171s
0.3038s
2.5339s
2.2178s
2.7749s

delta
+85.2ms
+102.3ms
+24.2ms
+156.1ms
+121.1ms

delta per obj/rel
+0.009ms
+0.01ms
+0.01ms
+0.31ms
+0.12ms

We cannot ignore the bug either, though. How about a compromise of making a new minor release (instead of a patch) and aiming to replace the solution by a more performant one?

How about a compromise of making a new minor release (instead of a patch) and aiming to replace the solution by a more performant one?
What do you mean?

I'm saying our users need a fix for their problem, and that would normally mean releasing a version 0.11.1. This would mean worse performance for everyone who set their version constraint to 0.11.x.
If we instead release a version 0.12.0, then those users will not accidentally incorporate performance degradations for no reason, and those who need the fix can use v0.12.0.

does this patch really fix problem? I actually never had any issues here, so not sure. I saw tests, is it reproducible?

But I'm for "minor version" and would be nice to write about that and performance decrease in changelog.
I will try to spend some time on weekends and revise session, maybe I will get solution. That part was second that was most annoying for me.

Yes, it is reproducible and this patch fixes the problem.
See https://travis-ci.org/haveyaseen/redux-orm/jobs/398190734.

Great to hear :slightly_smiling_face:  It would be interesting to see an alternative solution to the problem.

I'm not sure yet, but meanwhile - let's publish a minor version


Original WSI data

Sorry to interrupt, but your work is very innovative and provides a new dataset and report generation method for that dataset to the field of medical report generation. Can you make your WSI dataset public? Or be able to show a few images first to see what they look like. Thank you very much for your hard work!

Hello. Since the original WSI data are very large and take around 7T space, we are trying to organize them and find an online platform to upload them. A better way is to directly download them from the TCGA data portal. Meanwhile, the feature files we provided are ready to be downloaded, and you can use these files to reproduce our experiments (which does not need the original WSI files).


Feature/add command implementation

Closes #9
Implemented the Add Command, made the requested corrections. Can you shed light on what needs to be done on the toml file

Hey I want to try to stick to Friday releases if possible. This would be great to add to 0.0.4-alpha. If you don't mind I'll try helping you wrap this one up.
Thanks again for helping out!


Link broken for impact pdf download

Context
This page: https://2i2c.org/blog/2024/report-czi/ has a link to a pdf of the impact report https://2i2c.org/report-czi-2021/build/report-c7e3595ff6c09f5cc22cc52b8177cd4c.pdf That link gives a 404 upon a get request.
Proposal
No response
Updates and actions
No response

oh no that's super annoying - it looks like MyST hashes the filename of the PDF and so it broke when the site got re-built. We'll need to figure out how to generate that PDF link without the hash...it's programmatically generated each time we build the site which is why that's happening, thanks for catching!


A11y_Model Builder_Add data_Advanced data option_Info and relationship: After selecting any list item from dropdown then narrator is announcing previous selected list item details instead of updated.

GitHubTags:
#WCAG1.3.1; #A11ySev2; #A11yTCS; #A11yWCAG2.1; #DesktopApp; #Win11; #Visual Studio Client; #SH_ModelBuilder_Win32_May2023; #Info and Relationships; #Screenreader; #Narrator; #FTP; #Win32; #Benchmark;
Actual Result:
After selecting any list item from dropdown then narrator is announcing previous selected list item details instead of updated.
This issue observed throughout the application wherever the dropdown is present.
Issue not reproducible with NVDA.
Expected Result:
After selecting any list item from dropdown then narrator should be announce updated list item details instead of previous one.
Environment Details:
Application Name: Microsoft Visual Studio Enterprise 2022 (64-bit) - Current
Version 17.6.1
Microsoft Windows 11 Enterprise 10.0.22621 Build 22621
Screen reader: Narrator
NVDA version: 2020.4
Repro Steps:

Turn on Narrator.
Launch the Visual Studio with at least one of the .NET workload.
Tab till enable ML.NET Model Builder (Tools -> Options -> Environment -> Preview Features) and select it.
Create a new .NET project or open an existing .NET project.
Open context menu(Right-click) of the project in the solution explorer.
Tab till Add and then tab till Machine Learning Model and press enter.
Model Builder dialog will open. Complete Scenario and Environment setup steps.
Tab till value prediction tiles and press enter.
Add Data screen will appear. Tab till Advance data option and press enter.
Verify whether After selecting any list item from dropdown then narrator is announcing updated list item or not.

Verify whether the narrator is reading the tab items twice or not.
User Impact:
Screen reader user will not be able to get the proper information regarding the tab items.
Attachment:
Narrator:

NVDA:

The issue is not reproduced consistently, it seems that there is a time delay between the UI update and the narrator's announcement.

@SachinChavhan317 This is not reproducible with NVDA and  it seems that the narrator also narrate selected item sometimes hence it appears that the issue is specific to the narrator
are you able to down the sev of this issue?

@zewditu We cannot downgrade the severity for this issue. After selecting new item form the list, screen reader is announcing previous list item name even the selected list item got updated.
We can downgrade this bug to sev3, if screen reader doesn't announce previous selected list item name once new list item get updated. Thanks!

And I also noticed that the issue is not reproduced if the user selects the item using the mouse.

@SachinChavhan317  I think we need to close this issue, as per our discussion during accessibility office hours on 7/18/2023 with experts it is because of this known issue https://microsoft.visualstudio.com/OS/_workitems/edit/38960730.

As per discussion with SME, this is a narrator issue. Hence closing the bug.

#Regressed:07-19-23; #Closed;


lua plugin: update CMakeLists.txt for ts_lua_vconn.cc

Updating plugins/lua/CMakeLists.txt to build ts_lua_vconn.cc. Otherwise the plugin fails to find the symbols defined in there when the plugin is run.

👍


Allow ?ui=1 to override hiding of all desktop UI

For the Thimble case, we really want to hide the menu bar, sidebar, toolbar, etc.  But it would be nice if we could override this and allow the full UI to still be displayed.  Let's alter the brackets-browser-livedev extension to check for a ui=1 on the query string, and skip hiding UI elements when present.

Alright, I'll look into implementing this


Fix ash sanctifier

redo of https://github.com/oldschoolgg/oldschoolbot/pull/6061 since it affects osb & bso
Description:

Fix ash sanctifier not giving proper xp and removing ashes

Changes:

Added a check for multiplier by checking BOT_TYPE
pass updateBank to ashSanctifierEffect() to handle removing ashes
Fix monsterActivity & zalcano ashSanctifierEffect() usage
update zalcano return message to incorporate ashSantifierEffect()
update zalcano to use updateBank to handle giving loot

Other checks:

[X] I have tested all my changes thoroughly.

ashSanctifierEffect should instead take an updateBank, and add/change it, instead of making its own one.

ashSanctifierEffect should instead take an updateBank, and add/change it, instead of making its own one.

The issue is that updateBank.merge(ashSanctifierResult.updateBank); isn't sufficient at removing the ashes. ashSanctifierResult.updateBank has them removed but when you merge that with the original updateBank the ashes return.
you could do something like updateBank = ashSanctifierResult.updateBank; and make updateBank a let instead of const. Not sure if thats something you want to do though.

Seems like the code hasnt been linted. I always run yarn dev before committing, which I recommend.

Seems like the code hasnt been linted. I always run yarn dev before committing, which I recommend.

Fixed.


Independent splats in array pattern matching

Let's say that I want to get the first and last values of an array of arbitrary length, in a nice one-line statement. I could write
[first, middle..., last] = arr

But this is less than ideal from both a code efficiency standpoint and a readability standpoint, since I never use middle. What I'd prefer to write is simply
[first, ..., last] = arr

Do others agree that this should be allowed?

This all seems to be sorted out now on master.
list = [1..100]

[first, [], third] = list

console.log first, third

[first, []..., last] = list

console.log first, last

Produces...
1 3
1 100


Recently opened scripts temperature hard to read and other theme issues

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Windows 10 64Bit, current Godot 3.0 master
Issue description:
The recently opened scripts are now hard to read

Also the console gets spammed with errors (even when opening a fresh new project)

Though both problems have something to do with editor appearence (theme) I don't know if they are really related though.

Duplicate of #11059 #11095 #11174

I should include closed issues when I search for duplicates the next time before I open a new issue sorry! :D

No worries! :D


encryption: encrypt EncryptedContent by AES256-GCM

What problem does this PR solve?
Encrypt EncryptedContent by AES256-GCM.
Related changes

Need to cherry-pick to the release branch

Check List

Unit test
Integration test

Release note
Encrypt EncryptedContent by AES256-GCM.

/run-all-tests

/run-unit-test

/rebuild


Unable to build quickstart hello image

System Version: macOS 14.2.1 (23C71)
Kernel Version: Darwin 23.2.0
Model Name: MacBook Pro
Model Identifier: Mac14,6
Model Number: Z17900052LL/A
Chip: Apple M2 Max (ARM)
Total Number of Cores: 12 (8 performance and 4 efficiency)
Memory: 64 GB
I am trying to follow the steps in this quickstart guide: https://knative.dev/docs/getting-started/build-run-deploy-func
I have reached the build function image step, https://knative.dev/docs/getting-started/build-run-deploy-func, and I get the following error when I run kn func run --registry <registry-url>:
<registry-url> has been redacted on purpose.
 Any help you can provide would be much appreciated.

Have you changed module name in go.mod? @isc-rsaptars

The module must be named function, @isc-rsaptars .

Here is my go.mod file:
module function

go 1.21

I believe this is similar to https://github.com/knative/func/issues/2097 (which was resolved?)
And the issue stems from this error:
unable to satisfy target os/arch constraints; run image: {"os":"linux","arch":"arm64"}, buildpack: [{"os":"linux","arch":"amd64","distros":[{"name":"ubuntu","version":"18.04"}]}]
err<EMAIL_ADDRESS>

If I understand the error correctly, the error is due to the linux/arm64 buildpack not being present

Ah I see.

But I think currently is should work in emulation mode. Do you have qemu installed?

@isc-rsaptars I think paketo buildpacks might not support arm64 yet.

If I were to create a paketo-buildpack for arm64 following these instructions: https://github.com/dmikusa/paketo-arm64
Is there any configuration that I would have to do for it to play nice with knative func?

Optionally, are there any instructions on how to build images using the -b s2i option?

If I were to create a paketo-buildpack for arm64 following these instructions: https://github.com/dmikusa/paketo-arm64 Is there any configuration that I would have to do for it to play nice with knative func?

You might try setting builder image and buildpacks in func.yaml.

The s2i build strategy is multi-arch however we do not support that for Go, it should however work with the other languages.

If you find some other buildpack builder that works for arm64 you can use it by --builder-image={YOUR IMAGE HERE}.

Thanks! I will keep you updated on how that goes

@matejvasek looks like paketo-buildpack does support arm64 now: https://github.com/paketo-buildpacks/jammy-tiny-stack/pull/108

How is the ghcr.io/knative/builder-jammy-tiny image built?

@matejvasek looks like paketo-buildpack does support arm64 now: paketo-buildpacks/jammy-tiny-stack#108

That's just a stack image not builder/buildpack images.

paketobuildpacks/builder-jammy-tiny:latest image is not multi-arch.

The module must be named function, @isc-rsaptars .

Hi @matejvasek ,
Can share me the doc which can help me clear my doubts on this?
What's reasoning behind this?
Thanks

Here is my go.mod file:
module function

go 1.21

I believe this is similar to https://github.com/knative/func/issues/2097 (which was resolved?)
And the issue stems from this error:
unable to satisfy target os/arch constraints; run image: {"os":"linux","arch":"arm64"}, buildpack: [{"os":"linux","arch":"amd64","distros":[{"name":"ubuntu","version":"18.04"}]}]
err<EMAIL_ADDRESS>

I am facing same issue but in my case, there is no error?
unable to satisfy target os/arch constraints; run image:
Arch: amd64

The module must be named function, @isc-rsaptars .

Hi @matejvasek ,
Can you share me the doc which can help me clear my doubts on this? What's reasoning behind this?
Thanks

It's just way things were implemented. The package is expected to be just function anything else will not work.

@lkingland could you please look at this? I do not own arm machine.


Wrong image capture in Chapter 4, section Check the agent's data store.

In chapter 4. Configure the agent to answer blood eligibility FAQs, section Check the agent's data store the image currently indicates that the source is a website (Australian Red Cross Lifeblood) but in the steps before we added the information from a GCS bucket. Therefore, the image should show the Unstructured Documents selection instead.

The image has been corrected and updated in the codelab. Thanks so much for letting us know!


wip: packagevariant controller: support namespacing target packages

Not yet ready for review.
Related issue: https://github.com/GoogleContainerTools/kpt/issues/3488
Adds the set-namespace function to target downstream packages using the package-context pattern. This should result in the target packages' resources ending up with the specified namespace set on them.

Thanks Natasha. I don't necessarily want to tie this directly to the package context pattern. In fact, I think that pattern is already supported without this change. If the user sets the package name as different from the upstream, and adds set-namespace in the package Kptfile, I think the existing code should already handle this.
The package context pattern for set-namespace is really around packages whose purpose is to provision namespaces - that is, the package name matches the namespace name. The idea of putting namespace in here is for packages that need to be installed in a namespace, we are making namespace and explicit input to the package variant. It's really a special case of general "mutation" that we are building in directly since it is so common.
Along those lines, I have been talking pretty extensively with the Nephio team on what we need from PackageVariant and would like to evolve the API a bit. I will put in a PR for a design proposal that evolves this to the next step, and then once we all agree I'll work on implementation along with whoever is able to contribute.

Thanks for your feedback @johnbelamaric! If I am interpreting your comment correctly, this PR is not needed and you are planning to submit a design proposal for api changes that you/Nephio need.
In that case, I'll close this PR, focus on getting https://github.com/GoogleContainerTools/kpt/pull/3764 merged, and await your proposal.

Oh, I didn't actually mean that, but I think it's OK to pause on this PR until the proposal is ready and we can debate it.


CLIMATE-672 - Update the input validation function for OCW datasets

ocw.dataset._validate_inputs considers datasets on curvilinear grids

Merged build triggered. Test Failed.

Merged build started. Test Failed.

Merged build finished. Test Passed.


large model training loss nan

Hi, thanks for sharing the code.
I would like to know how many GPUs do you use to train the large model on the waymo data set? Using tools/cfgs/waymo_models/voxelnext_ioubranch_large.yaml, I use 3090 GPUx4  with mixed-precision training, but the model has a problem of loss nan at the beginning,
I'm thinking is it because my batch-size is too small? or the learning rate 0.003 is  too large? or beacause of use-amp?( I have tried to decrease the learning rate, close the mix-precision training, but it didn't work)
so could please provide a pretrain-model if it is possible?

Hi,
I think the reason might be from mixed precision. In addition, please contact me via-email. I will send you the checkpoint.
Regards,
Yukang Chen


Fix exception thrown when offsetWidth/offsetHeight is 0

In unit tests, I get an exception because dom.offsetHeight is 0, scaleY turns to infinity and some calculated values become NaN. This PR sets scaleX and scaleY to 1 if it would otherwise be inifitity.
I think this should be fixed since other people running unit tests would hit this issue and it's not trivial to figure out why the problem exists.

So your getBoundingClientRect is some kind of stub that returns a zero-size rectangle? Because logically, it seems that if offsetWidth and offsetHeight aren't zero (which is checked above this code), the client rectangle should also have a size. What kind of DOM implementation are your tests using?

I'm using JSDOM

I guess this is non-invasive enough. Merged.


feat: add EUPHORIA chain config and tAURA warp token

Description

Register information of Euphoria Testnet
Added warp information of tAURA token from Holesky to Euphoria

Backward compatibility
Yes
Testing
Manually tested using Hyperlane CLI

please add a minor changeset using yarn changeset add


IIS Rewrite rules not working when published to GCP

I am publishing to WinServer 2016 Compute Engine.  My redirect rules from HTTP --> HTTPS are not working.  The site still allows HTTP.
Note: HtttRedirect and URL ReWrite 2.1 are installed on the server.  -- See attached files.

My full web-config (from VS2017) is

Is it possible that the Google Cloud Tools for Visual Studio is not copying web.config to GCP; or GCP is not handling the ASPNetCore IIS Integration?  My Program.cs is setup to use IIS Integration as shown below:
var host = new WebHostBuilder()
       .UseKestrel()
       .UseContentRoot(Directory.GetCurrentDirectory())
       .UseIISIntegration()
       .UseStartup<Startup>()
       .Build();

        host.Run();

@samgithub-duplicate Can you provide some more context on how you are deploying?  Google Cloud Tools for Visual Studio should be copying over the web.config.
The extension should only allow deploying ASP.NET applications to GCE not ASP.NET Core.

Yes I am using the extension to deploy the web "application" (written using ASPNET Core 2  -- very "similar to the "hello world sample."   To deploy, I am right-clicking on the application (in VS)) and selecting "Publish to Google Cloud".  The deployment works and things work when deployed, except for the HTTP --> HTTPS rewrite.
Do you happen to know if anyone has successfully deployed the "Hello World" app, configured IIS in web.config to rewrite all access to HTTPS?

A quick way to confirm my issue is to copy the web.config I sent to the hello world app, deploy it and see if it rewrites HTTP ---> HTTPS
I reached out to your team because I am at my wits end, having spent this entire week trying to resolve this.
Note:  I have  google-managed-certificates (Let's Encrypt) installed.

@samgithub-duplicate Thanks for the added context.
I'm OOO most of the rest of the week but I will try and look into this more next week.

@samgithub-duplicate I just did a quick deployment and check the machine after deployment and I am seeing the copied over web.config file.
Can you confirm that the file is actually not being copied over in your case?
Can you send me your http -> https rewrite so I can test it?

Here is my web.config:

webconfigWithURLRewrite.txt

@samgithub-duplicate It looks like these files are being copied over.  This doesn't seem to be an issue with the extension.
My best guess is this is an issue with SSL certs.  Have you installed/configured them on this machine?
If you are still having issues it may be worth contacting support


Trigger v-on:focus of bs-input?

Applying v-on:focus to a bs-input component doesn't work because (I assume) the parent element is a div with the class of form-group.
Is there a workaround to allow for a the following code to be valid for the input inside the bs-input component?
<bs-input label="Input Field" v-on:focus="setFocus"></bs-input>

Effectively, this component contain all the bootstrap input element, a div that include the input, header and messages, and focus don't work in a div element.
I added onfocus attribute, check the last update (v1.1.2)

Thanks @wffranco, that helps a bunch. Could we also get an onblur attribute added as well? That would round off the functionality of that.

Ultimately, what I think would be helpful here would be to create a data attribute for bs-input called "isFocused" that is toggled to true onfocus and toggled to false onblur.
This would easily allow for the toggling of form styles when a field is selected, such as though v-bind:class="{ 'focused': isFocused }". Toggling a "focused" class for the form-group div would be a great native feature of this component.
(Which is what I am ultimately trying to do here while using the bs-input component--make it so that when the form is focused, the style and location of the label changes by adding a "focused" class alonside "form-group".)

I created a pull request, creating onblur and the focused attribute/state: https://github.com/yuche/vue-strap/pull/309


Migrate to Capacitor 5

Updates the plugin to support Capacitor 5.
I've confirmed this is working for Android and iOS, tested in the simulator and through npm run verify

Version 1.x.x is compatible with Capacitor 5
Version 0.6.0 is compatible with Capacitor 4

I've confirmed this is working on iOS and Android, with sources 0 (DEFAULT) and 2 (STATIC), I'm not sure why I wasn't seeing source 1 (REMOTE), but I don't believe it's due to any issue with the implementation we've got (I'm connected to a development environment and the remote config value only has a default value)


Podcast description text block is not anchored to its parent

Describe the bug
The podcast details page shows the podcast description, which can be expanded or collapsed if necessary; however, the text block is not anchored to the top of its parent, so the margin at the top of the text block can vary between podcasts. This is particularly noticeable at lower resolutions.
To Reproduce
Steps to reproduce the behavior:

Open a podcast.
Open a second, third etc and see the margin at the top of the text block change.

Expected behavior
The text block should be anchored to the top of its parent to keep the margin and padding consistent.
Screenshots

As I understand, the margin above the description text is part of the HTML renderer itself. Most show descriptions are in plain text (as opposed to episode descriptions). HTML and plain text are prestyled differently.

Here "Add to playlist" has wrapped its description inside  tag. There is added vertical margin to  tags by default. On the contrary, "2.5 Admins" has plain-text description and the absense of a  tag can be seen in the reduced margin above the text field.
Fixing this will require setting  tag margins to 0, but there can be other tags with default styling like  title tags. I have rarely seen tags other than  in the show description. I will include a fix in #123


Module 8/Exercise 1/Task 1/Step 7

Module: 08
Exercise: 01
Task: 01
Step: 07
Following line needs to be modified as follows:

Click Review + Create and then Create.

the Azure Firewall not work. When is check access from IE in Srv-Work, all address are unavailable and DNS can resolve the name www.bing.com and www.microsoft.com


chore(build): Fix typo and new pre-commit config

Fix agent typo
Fotmat a part of code(include: dbgpt/core, dbgpt/agent, dbgpt/serve, examples)
Add new pre-commit config

Please run following command to install new pre-commit hooks
pre-commit install

Every time you execute the command github commit xxx it will automatically trigger make fmt and make test
And you can manually trigger git hooks through the command pre-commit run --all-files


Added rewrite anchor target filter

Is it cool if I submit a new filter?
Added AnchorTargetFilter:
# HTML filter that rewrites 'target' attribute to all anchors in a document.
# Useful for causing to open in a new tab (set target to '_blank').
#
# Context options:
#   :target - The target to be added to <a>, defaults to TARGET ('_default')
#                Set to 'false' to remove targets from anchors.
#

Hi, thanks for the Pull Request. This is cool!!! But I am not sure if it is common enough to belong in html-pipeline. @simeonwillbanks What do you think?

@audy Thanks for the Pull Request!!!
I agree with @JuanitoFatas.  Please, create an html-pipeline extension, and we can link your AnchorTargetFilter extension in README.md. :smile:

Okay. Thanks!


v18 is for Victoria

Dedication

Victoria Carmen White, a black trans woman, was murdered in New Jersey, US, on 12 September 2010. Her boyfriend was brought to trial for her death, but he was acquitted and blamed another man for the murder. The gun used in the killing was never found.
Transgender women suffer too frequently from transphobic violence and cruelty, being the less protected social group. I'd like to raise an awareness of this problem. Humans should be creators — not killers. But most importantly, I want every transgender girl to have an opportunity to create applications quickly and, in general, learn to write code easily in order to receive job offers and leave dangerously transphobic territories for more favorable and civilized ones, and live happily there. Protect transgender women.

Version 18
Description
This version is focused on making the quick start with Express Zod API even faster by replacing Winston with a built-in console logger having the same basic functionality needed to begin the development. Migration should be easy in most cases.
What's included

#1636
#1640, later overridden by:
#1654
#1671

QA passed in general, but a minor thing I'd like to fix is color detection of ansis when running as PM2 process.

QA passed ✅


Fix Firefox not building

The current recipes try remove a not existing file (blocklist.xml) and fails, commenting the line solves

Thanks @sudo-give-me-coffee


Please update your PREFERRED_PEERS and KNOWN_PEERS settings in configuration file

Please update your PREFERRED_PEERS and KNOWN_PEERS settings in configuration file
Will you update?

I checked, and settings are up to date in current master


font weight bold for ActionList.Item with inline description

When an inline description is added to the item the normal text gets bold. This is how it is designed and specified in the docs: https://primer.style/design/components/action-list
Screenshots

Before
After

Merge checklist

[x] Added/updated tests
[x] Added/updated documentation
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

@colebemis how do I update the screenshots so that the vrt runner does not fail?


Create anomaly detection workflow

Summary
We should perform better data quality checks, and ideally have tools to help us identify data anomalies, as part the normal flow of our data work.
Problem
We currently perform sanity checks via assertions or with ad-hoc code outside of ETL, or via visual inspection on Indicator Upgrader explorer tool, or chart diff. But many data issues still remain in our data, and often users point them out.
In most cases, the issues were "not our fault", since they were already in the original data. However, we should, at least, be aware of these issues, contact data providers early on, and fix them when possible.
Impact
These data issues can lead to the following undesired outcomes:

It's time consuming for data managers to add ad-hoc sanity checks
Users become aware of data quality issues that we didn't spot first, which erodes trust
We miss spurious data points that we might otherwise have removed, meaning spurious trends may be picked up and amplified by our users

Scope

[ ] #3369
[ ] #3370
[ ] #3371

Notes taken during our workshop in the offsite

Dataset Quality Checker Proposal
This is a rough idea for a basic dataset quality checker. The goal is to see how technically challenging it might be while helping us spot important anomalies we can actually do something about. We're looking for things like fat-finger errors, out-of-range values, or extreme differences between countries. We're not trying to catch more complex stuff like regime shifts or changes in definitions—those are a bit too advanced for now.

Calculate statistics:
For each indicator in the dataset, let's calculate some basic stats—min, max, mean, etc. We'll also look at differences between data points to catch any sudden jumps. Maybe we'll throw in some other random checks to see what sticks.

Add Some Context:
Attach relevant metadata for each indicator (like unit, description, etc.) so we can better understand the numbers.

Ask ChatGPT:
Feed all this info into ChatGPT and ask it to flag any weird indicators or countries. While we're at it, we can also ask for countries that might be useful for comparison.

Visualize the Results:
Once we have the flagged data, let's plot it as charts so we can easily see what's going on.

This could be relatively easy to pull out for a few datasets and see how well does it work. Ideally, we'd have a list of real anomalies to test it against, but if not, we can just artificially mess up data.

Proposal A
Summary
We create a new dedicated page in wizard for anomaly detection. To begin with, this will be a table with one row per indicator, among the list of indicators from datasets that have changed in a given PR.
Changes in wizard
We need a new dedicated page for anomaly detection.
New backend code for anomaly detection in wizard
We need the code to execute anomaly detection at the individual indicator level, and when comparing indicator versions.
Changes in indicator upgrader (optional)
We may also need to look into how to handle variable mappings. In the easiest version, we just store a json file locally, but in the long term it would be good to be able to "restore" mappings in indicator upgrader, and store mappings in the grapher database.
Distribution of tasks
We thought we could roughly divide the work as follows:

@lucasrodes can take care of creating the streamlit app.

Lucas can also think a bit more about how to handle variable mappings, and whether the changes in indicator upgrader are necessary.

@Marigold can take care of indicator-related anomaly detection, possibly using LLMs.
@pabloarosado can take care of the indicator versions comparison, e.g. comparing each of the old and new indicators in an updated dataset, and detecting abrupt changes.

To add on Pablo's summary:

An extra motivation for working on something that keeps variable mappings is that we could use this information for other projects. Relating old → new indicators could enable things like pointing users to older indicator versions on our site, etc.

From our discussion:
First version

In the Wizard, there is a page for anomaly detection
The page should present you with a list of all changed grapher indicators

You may optionally ask about any arbitrary ETL path for a grapher indicator

You click "Detect all anomalies"
It begins fetches data from the S3 API and finds things whilst you wait

It should be interactive speed, i.e. take less than one minute
(optional) It would be dreamy if they appear one by one whilst you wait

The anomaly detector tries to return only the few "most important" anomalies
It stores them in some YAML file or SQLite DB; they're still there if you come back later
There is a button to regenerate them all
There is a button to delete them all
They are ephemeral: when you merge your branch, and your staging server dies, they disappear

But you can still manually add something important to "What you should know about this indicator" for its data page

Later options
If all of this went well, we could consider:

Automatically trigger this on change (e.g. on push), or otherwise do them as a batch
Some kind of workflow where we save the most important stuff into metadata
Some kind of visualisation that can show you the chart with the anomaly

Today we had a long discussion where we discussed which are the tasks pending and how to organize ourselves to work on this project.
From our meeting, we concluded that there are three main parts:

Backend: Working on a rule-based anomaly detection tool for single indicators and indicator upgrades.
Backend/DB: Make sense of the detected anomalies, group similar ones, and store them in a database.
UI: Present the relevant anomalies to the user in a streamlit app.

Some other important points:

We might need pagination
Review/Pending anomaly status is just optional (we don't force users to review, like in chart diff)
We need filter options to show more/less anomalies, show certain anomalies, etc.

Other pending discussion points

Agree on each phase's input and output specs (1, 2, 3).
It needs to be clarified what gets saved in the database.

Figma board

The first version of The Anomalist has been merged into the master and shared with the team. Pablo shared it with a short demo on how to use it.
We are currently gathering enhancements and improvements in this separate issue: https://github.com/owid/etl/issues/3436
Unsure if this issue should remain open. thoughts @pabloarosado ?

Thanks @lucasrodes, I think we can close this "big project" issue for now, and keep the tracking issue open.


Feedback for “Experimental”

In Vite.js, I enabled auto mode, but even after adding the noslot code in my APP.jsx file, the slot tag persists when I use it. How can I resolve this issue?
The issue is affecting my UI and CSS styling. because of Slot tag

In Vite.js, I enabled auto mode, but even after adding the noslot code in my APP.jsx file, the slot tag persists when I use it. How can I resolve this issue?
The issue is affecting my UI and CSS styling. because of Slot tag

Hmm, at the moment the experimental... option is very tricky as it isn't completely stable. However, I would suggest 2 things:

Move the experimental option to the root of the project in say your index file.
Try using the million-ignore in those files where you are experiencing this.


Can not drag on IE with touch screen

When using a win8 laptop with a touch screen (I imagine it is the same case with a win8 tablet), the library can not initiate the drag and drop on IE.
It works perfectly in chrome, In Firefox it works when you drag horizontally but if you drag vertically it scrolls the page. In IE it does not work at all.

Same here. Watched the code and I think you need the pointerdown MSPointerDown etc events to handle drag&drop on these devices.

Used pointer events for IE  and it triggers _onDragStart() and _onDrop() function but I am not able to drag items. If I press items for a long time, it triggers _onDragOver() function but not able to move them by dragging.

Have you tried using the ms-touch-action/touch-action css value?
Try this in your css and see if it fixes the issue:
	-ms-touch-action: none;
	touch-action: none;

It basically tells IE to turn off special touch handling for the element, which means the touch will go though as a click instead, and then most widget behavior then works.  Just something to try.
I included both versions as Microsoft has changed from the first to the second for IE11, but both will let you handle older IE versions.

This is also causing an issue for our solution. We've tried all of the above and other workarounds and nothing seems to fix the issue. Everything works in IE if it is only touch or only mouse. We see the issue when the device has mouse pointer and touch capabilities. Appreciate any movement on this issue or further workaround possibilities...


Unable to run Sample

Hi I am unable to run the sample here is the error i get, can somebody please help
{
"error": "unauthorized_request",
"error_description": "Unauthorized OIDC request"
}

@rjankathi Were you able to get the example working with the above advice?

Thank you Aaron I was able to successfully run it. I missed using the Web SDK and figured it out but thanks for the advise which exactly what was needed.

Glad you got it working.  Good luck and thanks for using Duo!


Reorder and hide columns within dataset viewer

Problem
When doing some basic vibe checks for datasets, I realized that the order in which columns are shown might not always be useful for viewing and exploring the data. I might want to quickly show chosen_model next to chosen_response and chosen_avg_rating and continue exploring based on that. Also, this representation might different based on personal preference and search session.
Related
I noticed an earlier discussion here https://github.com/huggingface/dataset-viewer/issues/2639 (thanks @davanstrien) but it does not fully align with that ideation.
Implementation
I see it more as an on-the-fly UI thing than a configurable Yaml thing (proposed in #2639), similar to re-ordering a tab in a browser. Based on this, I can also see an option to potentially hide useless columns.
Example
Based on UltraFeedback

cc @gary149 @krampstudio @julien-c: what do you think?


formatTime parsing incorrectly?

When the milliseconds parameter is less than 60 seconds, it returns ss, when it should return 00:ss because when the milliseconds parameter is 60 seconds or more, it returns mm:ss.

That's how the parser works but this in the rewrite will fix that.


unmet peer dependency "<EMAIL_ADDRESS>

Describe your environment

Operating System version: macOS Monterey v12.3.1
Firebase SDK version: v10.1.0
Firebase Product: firebase-admin
Node.js version: v17.5.0
Yarn version: v1.22.18

Describe the problem
Steps to reproduce:
I get the following warning when installing the latest version of firebase-admin (10.1.0)
warning "firebase-admin ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>

Same cause as https://github.com/firebase/firebase-js-sdk/pull/2082 and potentially same fix (inside the Firebase JS SDK). When transitioning from v8 to v9, app (now app-compat) was moved back into peerDependencies. I think the same conditions still exist such that we can move it back to devDependencies.

This should be fixed in https://github.com/firebase/firebase-js-sdk/pull/6319
Thank you!


Send notifications on refresh and begin write

This is a prerequisite for suppressing specific notifications from a write transaction, as we can't have notifications cover a range of commits with a gap, and generally makes it harder to get into inconsistent states.
The new behavior is that autorefresh works as it always did (advancing the most recent version with notifications ready), but now refresh() and begin_transaction() block if the notifications for the newest version are not ready yet. In the case of refresh(), the "newest version" means "the version that was newest at the time when refresh() was called or any version after that (which may still not be the actual latest version)". For begin_transaction() it blocks after acquiring the write lock, so it actually will be notifications for the latest version.
This makes refresh() within a notifications a no-op, and does not send notifications if begin_transaction() is called from within a notification, as I was unable to come up with a model for nested notifications which was clearly better than not doing them at all.
This does not attempt to make version advances from importing handover objects robustly produce notifications.

The new behavior is that autorefresh works as it always did (advancing the most recent version with notifications ready), but now refresh() and begin_transaction() block if the notifications for the newest version are not ready yet.

How is this different from the previous behavior? As I read the docs, the begin_transaction should already trigger any outstanding notifications:

Before beginning the write transaction, beginWrite updates the Realm instance to the latest Realm version, as if refresh() had been called, and generates notifications if applicable

Did we not send notifications covering all the way up to the latest version before?

We sent Realm-level and KVO notifications, but not collection notifications. This was an oversight that could be considered a bug.


Avoid the word "stabilize"

Builds upon https://github.com/adetaylor/rfcs/pull/13.
I know that's what we want to achieve, but it also carries a commitment that's not necessarily relevant to the RFC.
I'd also suggest that once you do open the RFC to the wider community, you rename the branch ;)

Thanks for the PR!
It's my first time going through the Rust RFC process, so I'm very happy to be guided about the best way to get there.
In the specific case of this RFC, it largely is about stabilizing something that already exists in unstable Rust - albeit with small changes. I am worried that if we're not explicitly saying "here are some small changes we want to make to pave the way for stabilization" it almost feels like we're taking credit for the feature which is already in unstable Rust. However I can see your point that it's not appropriate to make commitments about stabilization - I might have to make some more extensive changes to recast this RFC as "here are some small changes to the existing arbitrary self types feature" if we shouldn't explicitly cast the RFC as being about stabilizing the existing work.


Unable to find NSExtensionPrincipalClass (CustomKeyboard.KeyboardViewController) in extension bundle!

Error displayed at Keyboard switch in the simulator
OSLOG-CFE11529-C8AF-4CE1-8A81-D4B369FBE111 7 80 I 0 {}	Initialization successful

OSLOG-CFE11529-C8AF-4CE1-8A81-D4B369FBE111 7 80 L 0 {t:1694135487.996880,tz:480,tzDST:1,tid:0x1ac225,type:"Default",subsystem:null,category:null,offset:0x2bb74,imgUUID:"A878F8C4-0100-0000-0000-004000000000",imgPath:"/Applications/Xcode3.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 17.1.simruntime/Contents/Resources/RuntimeRoot/usr/lib/libMobileGestalt.dylib",procName:"CustomKeyboard",pid:85184,uid:501}	No persisted cache on this platform.

OSLOG-CFE11529-C8AF-4CE1-8A81-D4B369FBE111 7 80 L 1 {t:1694135488.284682,tid:0x1ac223,type:"Fault",subsystem:"com.apple.extensionkit",category:"default",offset:0x52dfc,imgPath:"/Applications/Xcode3.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 17.1.simruntime/Contents/Resources/RuntimeRoot/System/Library/Frameworks/ExtensionFoundation.framework/ExtensionFoundation"}	__extensionPrincipalClass != nil - /Library/Caches/com.apple.xbs/Sources/ExtensionFoundation_Sim/ExtensionFoundation/Source/NSExtension/NSExtensionSupport/EXConcreteExtensionContextVendor.m:109: Unable to find NSExtensionPrincipalClass (CustomKeyboard.KeyboardViewController) in extension bundle! Please verify that the extension links the required frameworks and that the value for NSExtensionPrincipalClass is prefixed with '$(PRODUCT_MODULE_NAME).' if the class is implemented in Swift.Error printed at Keyboard Switch:

Is the configuration correct?

Hi! Does it work in iOS 16? Does it work without KeyboardKit?

Closing this. Feel free to reopen.


DependencyCheck can be integrated with Azure Devops

DependencyCheck can be integrated with Azure Devops ? if yes please help us with some Documentation

https://www.google.com/search?q=dependency+check+azure+devops
https://marketplace.visualstudio.com/items?itemName=dependency-check.dependencycheck
https://github.com/dependency-check/azuredevops/blob/main/build/README.md

Thank you @jeremylong


install_dependencies ignoring virtualenv?

Hi
i might be missing some basic python and virtualenv understanding
or this is a bug in pybuilder
i use pybuild to create a simple package with a CLI script
I am running the pyb on a virtualenv (as it will be running from Jenkins)
i added all required dependencies:
@init
def set_properties(project):
    project.set_property("dir_source_main_python", "src/main/python")
    project.set_property("dir_source_main_scripts", "src/main/python/projectX/Apps")
    project.set_property("dir_source_unittest_python", "src/unittest/python")
    project.build_depends_on("mock")
    project.depends_on('GitPython')
    project.depends_on('python-jenkins')
    project.depends_on('ConfigParser')
    project.depends_on('click')

and the build is passing fine , all test executed etc
But when i try to run my script , its missing the click module
looking into the virtualenv , and it is missing th dependencies
running with -X revealed that all dependencies where skipped :

[DEBUG] Execution plan is install_dependencies
[INFO]  Building tufin_brancher version 0.0.1
[INFO]  Executing build in /var/jenkins/workspace/Test_python_job
[INFO]  Going to execute task install_dependencies
[DEBUG] Executing task 'install_dependencies'
[DEBUG] Executing action 'create_install_log_directory' from 'pybuilder.plugins.python.install_dependencies_plugin' before task
[DEBUG] Creating log directory 'target/logs/install_dependencies'
[DEBUG] Executing subtask from pybuilder.plugins.python.install_dependencies_plugin
[INFO]  Installing all dependencies
[DEBUG] Inspecting dependency 'mock'
[DEBUG] Dependency 'mock' is already up-to-date and will be skipped
[DEBUG] Inspecting dependency 'ConfigParser'
[DEBUG] Dependency 'ConfigParser' is already up-to-date and will be skipped
[DEBUG] Inspecting dependency 'GitPython'
[DEBUG] Dependency 'GitPython' is already up-to-date and will be skipped
[DEBUG] Inspecting dependency 'click'
[DEBUG] Dependency 'click' is already up-to-date and will be skipped
[DEBUG] Inspecting dependency 'python-jenkins'
[DEBUG] Dependency 'python-jenkins' is already up-to-date and will be skipped

Where\how does pybuild check for the dependencies ?
Does it suppose to work with virtualenv ?
Update :
I checked the same scenario on windows , and its working as expected (dependencies are installed on the virtualenv)

@tal5ab, could You please call pip list into virtual environment? How do You activate virtualenv for Pybuilder?
PS I use a huge of projects on virtualenv and Pybuilder and do not have problems.

Platform you're running on?
Version of PyB?
Version of Python?

Centos 6.9
Python 3.6

(.pyenv-usr-local-bin-python3.6) [root@trainlogic-centos Test_python_job]# pyb --version
pyb 0.11.10
(.pyenv-usr-local-bin-python3.6) [root@trainlogic-centos Test_python_job]# which pyb
/usr/local/bin/pyb

[root@trainlogic-centos Test_python_job]# pip3.6 list
DEPRECATION: The default format will switch to columns in the future. You can use --format=(legacy|columns) (or define a format=(legacy|columns) in your pip.conf under the [list] section) to disable this warning.
certifi (2<IP_ADDRESS>)
chardet (3.0.4)
click (6.7)
configparser (3.5.0)
coverage (4.4.1)
flake8 (3.4.1)
gitdb2 (2.0.2)
GitPython (2.1.6)
idna (2.6)
mccabe (0.6.1)
mock (2.0.0)
mockito (1.0.12)
multi-key-dict (2.0.3)
nose (1.3.7)
pbr (3.1.1)
pew (1.0.0)
pip (9.0.1)
pipenv (8.2.6)
pybuilder (0.11.10)
pycodestyle (2.3.1)
pyflakes (1.5.0)
pypandoc (1.3.3)
python-jenkins (0.4.15)
requests (2.18.4)
setuptools (32.3.1)
six (1.11.0)
smmap2 (2.0.3)
tblib (1.3.2)
unittest-xml-reporting (2.1.0)
urllib3 (1.22)
virtualenv (15.1.0)
virtualenv-clone (0.2.6)
wheel (0.30.0)
xmlrunner (1.7.7)
[root@trainlogic-centos Test_python_job]# source .pyenv-usr-local-bin-python3.6/bin/activate
(.pyenv-usr-local-bin-python3.6) [root@trainlogic-centos Test_python_job]# pip3.6 list
DEPRECATION: The default format will switch to columns in the future. You can use --format=(legacy|columns) (or define a format=(legacy|columns) in your pip.conf under the [list] section) to disable this warning.
pip (9.0.1)
setuptools (36.5.0)
wheel (0.30.0)
(.pyenv-usr-local-bin-python3.6) [root@trainlogic-centos Test_python_job]#

Can it be because pybuilder was installed outside of the virtualenv?
Can you share what steps you doing before running pyb?

@tal5ab You're totally rigth -  it's because pybuilder was installed outside of the virtualenv.
I install PyBuilder into each new virtual environment (because virtualenv also install new pip for each virtual environment).

Could You please try to create virtual environment with option --system-site-packages and try again?

For avoiding problems into virtualenv I use "scorched-earth policy" for Python:

alexeysanko@alexeysanko-VirtualBox:~$ pip list --format=columns
Package    Version

pip        9.0.1
setuptools 36.5.0
virtualenv 15.1.0
wheel      0.30.0

And install any packages into virtualenv. As I know it's recommended way of virtualenv usage (it allows to avoid any compatibility problems).

So , i ran pip3.6 uninstall pybuilder outside of virtualenv
And installed it back inside the virtualenv , and now its working as expected
Is that the way it should be working ? Install pybuilder on the new virtualenv every time ?
I need to add pybuilder installation to my jenkinsfile ?
I expected pybuilder to check dependencies on the virtualenv no matter where it was installed
If that is by design , please close this issue
Thanks
Tal

@tal5ab
I can tell only about my experience. We use next jenkins.sh for each jenkins dev task:
#!/usr/bin/env bash

# activate virtual environment
if [ ! -d ./venv ]; then
  virtualenv venv
fi
./venv/bin/activate
pip install --upgrade --upgrade-strategy "only-if-needed" pybuilder

# define pypi server
if [ -z "$PYPI_SERVER_NAME" ]; then
    PYPI_SERVER_NAME="default-pypi-dev"
    echo "PYPI_SERVER_NAME is unset. '${PYPI_SERVER_NAME}' will be used"
fi

# build project
pyb clean install_dependencies analyze publish upload --debug -P pypi_server=$PYPI_SERVER_NAME

# deactivate virtual environment
deactivate

We use our own pypi servers and need to specify it through pybuilder_pypi_server plugin.
For common usage it will look like
#!/usr/bin/env bash

# activate virtual environment
if [ ! -d ./venv ]; then
  virtualenv venv
fi
./venv/bin/activate
pip install --upgrade --upgrade-strategy "only-if-needed" pybuilder

# build project
pyb clean install_dependencies analyze publish upload --debug

# deactivate virtual environment
deactivate

just saw your comments , thanks
i will do that approach
Thank you for your time


/token endpoint CORS issue

Still getting CORS errors with PKCE flow after calling client.exchange.
Access to fetch at 'https://<redacted>.lambda-url.us-east-1.on.aws/token' from origin 'http://localhost:5173' has been blocked by CORS policy: The 'Access-Control-Allow-Origin' header contains multiple values '*, *', but only one is allowed. Have the server send the header with a valid value, or, if an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

@thdxr think this could be caused by the sst component because I think the underlying router sets wildcard cors headers as well?

i made an update to the Auth component to not set cors automatically (was being done by lambda function url) - but i have not tested it yet

Saw that. I've upgraded my sst and seems to be working for me. Thanks.

I still keep running into weird cors issues:

Access to fetch at '.lambda-url.us-east-1.on.aws/token' from origin 'http://localhost:5173' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Steps to reproduce:

run sst dev
authorize the user
call client.refresh(refresh, { access }) -> no cors error
(keep the browser tab open)
restart sst dev
call client.refresh(refresh, { access }) again -> cors error

I was sometimes able to reproduce this by making changes to my code while sst dev was running but not as reliably as the steps above.

this should be fixed in latest sst - leave a commennt if not


🔀 :: (#112) 유저 급식패스 정보 가져오기

Codecov Report

Merging #122 (007ce81) into develop (c5af2eb) will decrease coverage by 3.05%.
The diff coverage is 6.06%.

:exclamation: Current head 007ce81 differs from pull request most recent head 3d4b546. Consider uploading reports for the commit 3d4b546 to get more accurate results

@@              Coverage Diff              @@
##             develop     #122      +/-   ##
=============================================
- Coverage      37.96%   34.90%   -3.06%
+ Complexity        84       83       -1
=============================================
  Files             52       55       +3
  Lines            540      573      +33
  Branches          24       27       +3
=============================================
- Hits             205      200       -5
- Misses           325      362      +37
- Partials          10       11       +1

Impacted Files
Coverage Δ

...om/semicolon/gspass/controller/UserController.java
0.00% <0.00%> (ø)

.../com/semicolon/gspass/dto/user/GsPassResponse.java
0.00% <0.00%> (ø)

...ava/com/semicolon/gspass/entity/gspass/GsPass.java
0.00% <0.00%> (ø)

.../gspass/exception/GsPassAlreadyApplyException.java
0.00% <0.00%> (ø)

...olon/gspass/exception/GsPassNotFoundException.java
0.00% <0.00%> (ø)

...m/semicolon/gspass/facade/user/UserFacadeImpl.java
0.00% <0.00%> (ø)

...semicolon/gspass/service/user/UserServiceImpl.java
50.00% <0.00%> (-16.04%)
:arrow_down:

...om/semicolon/gspass/error/exception/ErrorCode.java
93.75% <100.00%> (+0.89%)
:arrow_up:

...colon/gspass/service/school/SchoolServiceImpl.java
77.50% <0.00%> (-8.75%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5af2eb...3d4b546. Read the comment docs.


Short solution needed: "How to use datetime type" (python-pandas)

Please help us write most modern and shortest code solution for this issue:
How to use datetime type (technology: python-pandas)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to use datetime type